Skip to content

Conversation

@elliot-choic
Copy link
Contributor

No description provided.

@netlify
Copy link

netlify bot commented Aug 21, 2025

Deploy Preview for ai-agent-messenger-sample canceled.

Name Link
🔨 Latest commit 2626ebc
🔍 Latest deploy log https://app.netlify.com/projects/ai-agent-messenger-sample/deploys/68fedcc5d11a0100088046dd

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR updates string resources in the Android application, primarily focusing on improving conversation-related UI text and adding new localization strings.

  • Updates conversation-related text labels for better clarity
  • Removes unused conversation action strings
  • Adds new UI elements for conversation status, timestamps, and agent interaction

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

<string name="aa_text_csat_submit">Submit</string>
<string name="aa_text_csat_dialog_edittext_hint"> Share your feedback</string>
<string name="aa_text_csat_notice_conversation_closed">Your conversation has ended.</string>
<string name="aa_text_csat_notice_conversation_closed">This conversation has ended.</string>
Copy link

Copilot AI Aug 21, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[nitpick] The change from 'Your conversation has ended' to 'This conversation has ended' appears subtle but could impact user experience. Consider whether this change maintains consistency with other user-facing messages in the application.

Suggested change
<string name="aa_text_csat_notice_conversation_closed">This conversation has ended.</string>
<string name="aa_text_csat_notice_conversation_closed">Your conversation has ended.</string>

Copilot uses AI. Check for mistakes.
Copy link
Contributor

@leo-shin leo-shin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@elliot-choic elliot-choic merged commit 698f9e3 into main Oct 27, 2025
6 checks passed
@elliot-choic elliot-choic deleted the feature/elliot/update-string-resource branch October 27, 2025 02:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants